Skip to content

chore: remove remaining navigator and nemoclaw references#279

Merged
drew merged 2 commits intomainfrom
chore/remove-navigator-nemoclaw-references
Mar 15, 2026
Merged

chore: remove remaining navigator and nemoclaw references#279
drew merged 2 commits intomainfrom
chore/remove-navigator-nemoclaw-references

Conversation

@drew
Copy link
Collaborator

@drew drew commented Mar 13, 2026

Summary

  • Rename NEMOCLAW_SSH_HANDSHAKE_SECRET env var to OPENSHELL_SSH_HANDSHAKE_SECRET in sandbox process/SSH code, E2E tests, and architecture docs
  • Fix stale crates/navigator-core/ path in .gitattributes (directory was renamed to crates/openshell-core/)
  • Remove commented-out NemoClaw row from docs/sandboxes/index.md

Known Remaining (intentional)

File Reason
crates/openshell-policy/src/lib.rs LEGACY_CONTAINER_POLICY_PATH = "/etc/navigator/policy.yaml" — intentional backward compat for old sandbox images
.github/workflows/publish.yml navigator-pypi-artifacts S3 bucket — already has TODO, requires AWS infra rename
python/openshell/_proto/navigator_pb2*.py Proto-generated files — need proto source rename upstream

@drew drew self-assigned this Mar 13, 2026
@github-actions
Copy link

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://NVIDIA.github.io/OpenShell/pr-preview/pr-279/

Built to branch gh-pages at 2026-03-13 09:21 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Rename NEMOCLAW_SSH_HANDSHAKE_SECRET to OPENSHELL_SSH_HANDSHAKE_SECRET
in sandbox code, tests, and architecture docs. Fix stale navigator-core
path in .gitattributes. Remove commented-out NemoClaw row from sandbox
docs.
@drew drew force-pushed the chore/remove-navigator-nemoclaw-references branch from e0f63ef to 7934790 Compare March 15, 2026 18:26
@johntmyers johntmyers self-requested a review March 15, 2026 19:43
@drew drew merged commit 9905a75 into main Mar 15, 2026
9 checks passed
@drew drew deleted the chore/remove-navigator-nemoclaw-references branch March 15, 2026 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants